Skip to content

[NO-MERGE][CMPT-5391] feat(pipelines): add dr pipelines run subcommand group - #3

Closed
sunny2get wants to merge 9 commits into
cmpt-5391/pipelines-crudfrom
cmpt-5391/pipelines-runs
Closed

[NO-MERGE][CMPT-5391] feat(pipelines): add dr pipelines run subcommand group#3
sunny2get wants to merge 9 commits into
cmpt-5391/pipelines-crudfrom
cmpt-5391/pipelines-runs

Conversation

@sunny2get

Copy link
Copy Markdown
Owner

NO-MERGE — Requirements are still evolving. This PR is open for early review only.
Stack: PR 1 (base)PR 2 (crud)PR 3 (this) → PR 4 → PR 5 → PR 6

Summary

  • Adds dr pipelines run create/list/get/status/cancel — trigger, inspect, and cancel pipeline executions
  • Wire-level API uses dispatches/dispatch_id; --output json remaps these to run_id/covalent_run_id
  • run status is a lightweight polling call; run cancel returns 409 if already terminal
  • Introduces cmd/pipelines/run/runutil/ rendering helpers and internal/pipelines/run.go

Stacked on: cmpt-5391/pipelines-crud — diff here shows only the delta.

Test plan

  • go build ./...
  • go test ./cmd/pipelines/run/... ./internal/pipelines/...
  • dr pipelines run --help — shows create/list/get/status/cancel
  • dr pipelines run create --pipeline <id> --input <input-id> --help

🤖 Generated with Claude Code

@sunny2get
sunny2get force-pushed the cmpt-5391/pipelines-crud branch from 179eb63 to 48d777e Compare May 25, 2026 17:17
@sunny2get
sunny2get force-pushed the cmpt-5391/pipelines-runs branch from 3b67874 to 8635c06 Compare May 25, 2026 17:17
@sunny2get
sunny2get force-pushed the cmpt-5391/pipelines-crud branch from 48d777e to 627d44e Compare May 25, 2026 17:40
@sunny2get
sunny2get force-pushed the cmpt-5391/pipelines-runs branch from 8635c06 to 3717d1b Compare May 25, 2026 17:40
@sunny2get
sunny2get force-pushed the cmpt-5391/pipelines-crud branch from 627d44e to 313694b Compare May 25, 2026 17:45
@sunny2get
sunny2get force-pushed the cmpt-5391/pipelines-runs branch from 3717d1b to d5887ee Compare May 25, 2026 17:45
@sunny2get
sunny2get force-pushed the cmpt-5391/pipelines-crud branch from 313694b to 77e7969 Compare May 25, 2026 17:56
@sunny2get
sunny2get force-pushed the cmpt-5391/pipelines-runs branch 4 times, most recently from c764828 to e8ffcc9 Compare May 27, 2026 15:57
@sunny2get
sunny2get force-pushed the cmpt-5391/pipelines-crud branch from 60edfb8 to af6597b Compare May 27, 2026 16:19
@sunny2get
sunny2get force-pushed the cmpt-5391/pipelines-runs branch from e8ffcc9 to 06be780 Compare May 27, 2026 16:19
@sunny2get
sunny2get force-pushed the cmpt-5391/pipelines-crud branch from af6597b to 6e509d6 Compare May 27, 2026 16:30
@sunny2get
sunny2get force-pushed the cmpt-5391/pipelines-runs branch from 06be780 to 3c836b4 Compare May 27, 2026 16:30
@sunny2get
sunny2get force-pushed the cmpt-5391/pipelines-crud branch from 6e509d6 to 81a676f Compare May 27, 2026 16:35
@sunny2get
sunny2get force-pushed the cmpt-5391/pipelines-runs branch from 3c836b4 to a540260 Compare May 27, 2026 16:35
@sunny2get
sunny2get force-pushed the cmpt-5391/pipelines-crud branch from 81a676f to 9382396 Compare May 28, 2026 14:37
@sunny2get
sunny2get force-pushed the cmpt-5391/pipelines-runs branch from a540260 to ade0a73 Compare May 28, 2026 14:37
@sunny2get
sunny2get force-pushed the cmpt-5391/pipelines-crud branch from 9382396 to 3b4d4c4 Compare May 28, 2026 14:40
@sunny2get
sunny2get force-pushed the cmpt-5391/pipelines-runs branch from ade0a73 to 18a5d09 Compare May 28, 2026 14:40
@sunny2get
sunny2get force-pushed the cmpt-5391/pipelines-crud branch from 3b4d4c4 to 6327eb4 Compare May 28, 2026 14:46
@sunny2get
sunny2get force-pushed the cmpt-5391/pipelines-runs branch 2 times, most recently from 8514908 to 5dfeca0 Compare May 28, 2026 15:30
@sunny2get
sunny2get force-pushed the cmpt-5391/pipelines-crud branch from 5322202 to a4d9113 Compare May 28, 2026 15:32
@sunny2get
sunny2get force-pushed the cmpt-5391/pipelines-runs branch 2 times, most recently from 53e1d22 to 8d9ca58 Compare May 28, 2026 15:54
@sunny2get
sunny2get force-pushed the cmpt-5391/pipelines-runs branch from 8d9ca58 to b6c7770 Compare May 28, 2026 15:58
@sunny2get
sunny2get force-pushed the cmpt-5391/pipelines-crud branch from 6393067 to 7e49fc5 Compare May 29, 2026 18:21
@sunny2get
sunny2get force-pushed the cmpt-5391/pipelines-runs branch from b6c7770 to 6a1de9d Compare May 29, 2026 18:21
@sunny2get
sunny2get force-pushed the cmpt-5391/pipelines-crud branch from 00e1376 to 051edb8 Compare June 1, 2026 14:11
@sunny2get
sunny2get force-pushed the cmpt-5391/pipelines-runs branch from 6a1de9d to 87b80fc Compare June 1, 2026 14:12
sunny2get and others added 8 commits June 1, 2026 10:58
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Adds the run subcommand group for triggering and inspecting pipeline
executions (create/list/get/status/cancel).

- internal/pipelines/run.go: CreateRun, ListRuns, GetRun, RunStatus,
  CancelRun — API client for the /dispatches endpoints
- cmd/pipelines/run/: run create/list/get/status/cancel + runutil output
- docs: run section in pipelines.md, run endpoints in pipelines-reference.md

Wire-level endpoints still use "dispatches"; CLI output uses run_id /
covalent_run_id per the vocabulary migration in the codebase.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Apply sunny/pipelines versions of run subcommand files and update
all import paths to use internal/pipeline (singular) after rename.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…n, limit defaults, version prefix

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@sunny2get
sunny2get force-pushed the cmpt-5391/pipelines-runs branch from 87b80fc to 9efa141 Compare June 1, 2026 14:58
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@sunny2get

Copy link
Copy Markdown
Owner Author

Superseded by upstream PR datarobot-oss#541: datarobot-oss#541

@sunny2get sunny2get closed this Jun 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant